Skip to content

brighter tab contrast for usability#762

Merged
1 commit merged into
mainfrom
brighter-tab-contrast
May 27, 2026
Merged

brighter tab contrast for usability#762
1 commit merged into
mainfrom
brighter-tab-contrast

Conversation

@5byuri

@5byuri 5byuri commented May 27, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings May 27, 2026 10:40
@5byuri 5byuri changed the title bg-background > bg-secondary brighter tab contrast for usability May 27, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adjusts the shared Radix Tabs UI styling to improve active-tab visual contrast, which impacts tabbed navigation across multiple pages/components that import TabsTrigger from src/components/ui/tabs.tsx.

Changes:

  • Changed the active tab background from bg-background to bg-secondary to increase visual separation between active and inactive tabs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ref={ref}
className={cn(
"inline-flex cursor-pointer border border-transparent data-[state=active]:border-border dark:data-[state=active]:border-transparent items-center justify-center whitespace-nowrap rounded-sm px-3 py-1.5 text-sm font-medium ring-offset-background transition-all focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 disabled:pointer-events-none disabled:opacity-50 data-[state=active]:bg-background data-[state=active]:text-foreground data-[state=active]:shadow-sm",
"inline-flex cursor-pointer border border-transparent data-[state=active]:border-border dark:data-[state=active]:border-transparent items-center justify-center whitespace-nowrap rounded-sm px-3 py-1.5 text-sm font-medium ring-offset-background transition-all focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 disabled:pointer-events-none disabled:opacity-50 data-[state=active]:bg-secondary data-[state=active]:text-foreground data-[state=active]:shadow-sm",
@timbastin timbastin closed this pull request by merging all changes into main in 3a90654 May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants